Skip to content

Implement WinUI3 support for Desktop package #5411

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 54 commits into
base: main
Choose a base branch
from

Conversation

DharshanBJ
Copy link
Contributor

@DharshanBJ DharshanBJ commented Jul 29, 2025

This pull request addresses the business need outlined in AzureAD/microsoft-authentication-library-for-dotnet#4933. The proposed changes are based on the design detailed in the following document: AuthLibrariesApiReview#18287.

I had shared the preview packages in the issue discussion, many of them tested the packages and confirmed that it works for them - #4933 (comment)

@bgavrilMS
Copy link
Member

Hi @DharshanBJ - this has been open for a while. Is it still relevant?

@DharshanBJ
Copy link
Contributor Author

Hi @DharshanBJ - this has been open for a while. Is it still relevant?

Hi @bgavrilMS , yes, this PR is still relevant, i'm waiting on reviews for this PR.

Copy link
Contributor

@ashok672 ashok672 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@bgavrilMS
Copy link
Member

@gladjohn - can you pls review this?
@DharshanBJ - I think this is all good, but there may be some changes on the build we need to do.

@DharshanBJ
Copy link
Contributor Author

@gladjohn - can you pls review this? @DharshanBJ - I think this is all good, but there may be some changes on the build we need to do.

@bgavrilMS , yes i'll be adding the build changes and public documentation as part of my next work items.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants